3to4: Fix suggested replacements for border-width

The new CSS border-spacing does what Grid::(row|column)_spacing and
Box::spacing already did, i.e. controlling the space added between child
widgets, so it’s not a replacement for Container::border-width.

combobox: Stop pointlessly NULL-checking the menu

We now have a valid menu to use at all times.

combobox: Move area and menu to .ui file

Now that priv->area is guaranteed to be constructed by us, and not
passed in by a user, we can move it to the .ui file and stop manually
managing its lifetime altogether. And once the area is there, we can
move the menu there too (and stop pointlessly destroying/rebuilding it).

combobox: Remove property cell-area

The CellArea is going to become a pure implementation detail & be moved
to the .ui, instead of letting users mess with it (if anyone ever did).

combobox: Reuse code rather than reimplementing it

We already have cell_layout_is_sensitive() to get whether at least one
cell in a Layout is sensitive, which we need because CellLayout/View
do not implement foreach(). So, since we wrote that, we can use it to
check our CellArea too, instead of doing foreach with a custom callback.

combobox: some code cleanups

 * Merge adjacent if statements with the same condition
 * Add a space between a function and its declaration

menushell: Fix select_last() to get last, not 1st

It was broken in the same way as select_first(), as explained in
commit a5211adb225172195d6778d652b58632c38afcbf

combobox: Fix documentation typo bx => box

menushell: Make select_first() do that, not last…

It was looping over all items, not breaking out when it found the first
selectable one, and then selecting the _last_ selectable one (if any)
found. So, it did exactly the opposite of its name. This made me quite
baffled when opening a submenu with right-arrow put me at its last item.

Originally, the loop set to_select and broke if the current item was
selectable and not tear-off, meaning that it would correctly select the
first suitable item. However, when tear-off functionality was removed
in commit 4ed9452e9028bb90589474783cebac57ab66e2c9, so was the break.

Update Polish translation

Fix CSS spelling in a translatable string

menushell: Don’t dereference before typechecking

Update POTFILES.in and POTFILES.skip

combobox: If typechecking, don’t reference before

Don't hide windows and popovers in init()

Instead, don't make them visible in gtk_widget_init in the first place.

fontbutton: Be a GtkWidget

assistant: Don't forget to remove child widgets

toolitemgroup: Properly remove internal child widget

We now need to use gtk_widget_unparent and can't just destroy it.

popovermenu: Remove forall implementation

button: Don't explicitly show child widgets

recorder: Adapt to widget visibility changes

We now need to explicitly hide the details label.

Fix some widget visibility fallout

widget: Remove show-all property

Doesn't make sense anymore now that gtk_widget_show_all is gone.

Remove gtk_widget_show_all

widget: Remove show_all vfunc

filechooserbutton: Be a GtkWidget

Instead of subclassing GtkBox to have 2 child widgets, subclass
GtkWidget and only render one of them.

sizerequest: Use gtk_widget_measure to measure widget sizes

gtkbuilder: Improve error messages

shortcutswindow: Unset search_bar in dispose

And check for null-ness in unmap. This prevents a crash when the window
is first disposed and then unmapped.

shortcutswindow: Use GtkButton:icon-name for search button

shortcutsshortcut: Be a GtkWidget

Instead of subclassing GtkBox and overriding _add to not work, subclass
GtkWidget and add a normal GtkBox as a child widget.

gtkshortcutswindow: Adapt to GtkWidget visibility changes

widget-factory: Remove some gtk3-widget-factory leftovers

filechooser: adapt to GtkWidget:visible changes

widget-factory: Adapt to GtkWidget visibility changes

Make the infobar and the toolbar invisible by default and remove all the
lines showing the other widgets.

coloreditor: Hide overlay boxes

Hide popovers and toplevels by default

widget: Be visible by default

combobox: popdown() the menu during unmap()

combo_box_popdown() currently skips popping down our menu if it is NULL.
But the required call to this at end-of-life was in destroy(), by which
point dispose() already NULLed the menu, so Menu::popdown() would never
run, even if it should. Fix this by trying popdown() earlier in unmap().
Also, add a converse assurance that we don’t popup() while not mapped.

Even once we remove all the now-pointless NULL checks, destroy() was the
wrong place to call combo_box_popdown(), and unmap() is the right place.

Revert "wayland: Push NULL buffer when hiding a GdkWindow"

This reverts commit 6beb0b91c91dd4a018b0fe2c39f5687a00938821.

update zh_CN translation

gtkmain: Fix build on Windows

gtk_init() removed its support for supporting arguments, so we ought to do
likewise for Windows, which actually defines items that call gtk_init()
the old way (and also get rid of argument support in those functions,
since the direction is to not support them).

https://bugzilla.gnome.org/show_bug.cgi?id=773299

Visual Studio builds: Update GSK project

Another GSK Vulkan renderer source was added for the build, so include it
here.

gtk: Register GSK resources at startup

That way they show up in the resource list in the inspector even when
running with the Cairo renderer.

combobox: Remove dead code & unneeded type checks

Commit fdc0c6426b6fb751653f3a8536a344f69a6407ac removed the appears-as-
list style property, & hence the ability to put the ComboBox into list
mode – but it left behind a pile of hijinks that were only used in said
mode & so were now doing absolutely nothing. This commit deletes those.

While doing that, I got carried away…so this also stops pointlessly type
checking popup_widget, as that can never be anything but a GtkTreeMenu.
It still checks for NULL everywhere, which shouldn’t be needed, but (A)
this commit is already too big, & (B) simply removing such checks where
they _seem_ unnecessary causes bad times. I’ll puzzle through that later

testcombo—Don’t set appears-as-list style property

List mode no longer exists, so this had no effect.

combobox: Make wrap-width and grid mode work again

Commit fdc0c6426b6fb751653f3a8536a344f69a6407ac for removing (partly!)
appears-as-list also deleted the code that propagated wrap-width to the
TreeMenu and thus put us into “grid mode”. This restores that code.

And as Benjamin noted, calling check_appearance() here is wrong, so bye.

Add a section about initialization to the migration guide

Just the facts.

Update callers

Adapt all our tests and examples to the new initialization api.

Drop gdk init api

Supporting separately initialized gdk is just not worth it.

Drop arguments from gtk_init

This is our ultimate goal: no more commandline argument handling.
Lots of callers will have to be updated.

Simplify initialization code

Now that gtk_get_option_group and gtk_parse_args are no longer
public, we can get rid of them.

Drop the --gtk-debug and --gtk-no-debug options

We have environment variables that cover this.

Drop the --g-fatal-warnings argument

We don't need it, GLib has an environment variable for this.

Drop the --gtk-modules argument

We don't need it; the environment variable is enough.

Don't export gtk_get_option_group

We want to get rid of commandline option handling in GTK+.
This is a step in that direction.

Stop exporting gtk_parse_args

We want to stop handling commandline arguments, and that is the
sole purpose of this function. So it has to go

Drop gtk_init_with_args

We want to simplify our initialization code and remove all commandline
argument handling from it. The first stop for this is to reduce the
number of gtk_init variants we have.

gdk: Skip GdkVulkanContext methods when building introspection data

We cannot use Vulkan types through introspection anyway.

wayland: Push NULL buffer when hiding a GdkWindow

This is how windows are meant to be hidden as per the wayland
protocol, there's no need to destroy the xdg_surface and other
interfaces.

Also, rename gdk_wayland_window_hide_surface() to clear_surface(),
as that's what it does.

https://bugzilla.gnome.org/show_bug.cgi?id=773686

combobox: Avoid a pointless assignment

Don’t get the active item pointer before the grid/non-grid conditional,
because if we’re in grid mode, we re-get it before selecting it anyway.

combobox: Also preselect active item in grid popup

i.e. when wrap-width > 0. This was only being done for non-grid cases.
So, ComboBoxes in grid mode did not indicate their selection when popped
up and required users to keynav from ‘nothing’ (at the top-left) to the
item they wanted to select. By selecting the active item in advance, now
it’s highlighted & acts as the starting point for keynav around the grid

scale: Really fix documentation of set_digits()

This previously only mentioned its effect on the displayed value, and
even after the previous commit, its rounding of the actual value upon
change still reads like too much of an afterthought. Worse, it wasn’t
mentioned at all in the doc for the @digits parameter. Change this to
emphasise rounding always occurs and the displayed value is secondary.

scale: doc: set_digits doesn’t round retroactively

Whether it should is an open question, but for now, the documentation
should clearly indicate that currently rounding is only applied upon
changes to the value, not to the existing value when ::digits changes.
This is already clear in the doc for the underlying Range::round-digits.

https://bugzilla.gnome.org/show_bug.cgi?id=358970

Scale: Always sync ::digits to Range::round-digits

The documents state that gtk_scale_set_digits() “causes the value of the
adjustment to be rounded off to this number of digits, so the retrieved
value matches the value the user saw.” Note the lack of any condition.

But in fact, if draw-value was false, rounding was disabled on the base
Range, so values that weren’t displayed weren’t rounded. This made the
docs wrong and made an apparently cosmetic detail alter functionality.

Fix by ensuring the number of digits set on Scale is always propagated
along to gtk_range_set_round_digits(), thus rounding to it in all cases
when the value changes, regardless of whether the value is displayed.

This doesn’t address the other idea from Bugzilla: that changing the
number of digits should clamp the _existing_ value if it’s more precise.
This contradicts digits docs in the base Range, but the above from Scale
can be read as implying it’ll happen. For now, that’s an open question.

https://bugzilla.gnome.org/show_bug.cgi?id=358970

combobox: Work around popup handler altering model

GtkFileChooserButton installs a handler for the popped-up signal, which
refilters the menu, in order to hide the “(None)” item from the popup
if it was previously selected in the ComboBox. This oddity means that:

 • Until recently, this item would be selected in the menu shell, which
   would then be popped up and change the selection away from that item.
   This was therefore redundant (more on which below!) but benign.

 • After the patch for https://bugzilla.gnome.org/show_bug.cgi?id=771242
   however, this causes a critical assertion fail, as now we stash the
   originally selected item in a pointer so that it can be selected only
   after realisation/popup – but by that stage, the model has just been
   refiltered and the previous pointer no longer refers to a valid item.

This commit works around this problem by, after popping up the menu,
getting the active item again, in case a popped-up handler has gone and
invalidated the pointer to the active item that we saved before popup.

If a handler does this, everything done to find/use the original item is
pointless. But this avoids the ugly critical in FileChooserButton, while
not harming every other ComboBox that doesn’t mess with its model while
popping up (hopefully the vast majority), and it’s very difficult to
imagine a way to check if the active item is /going to/ be hidden later)

combobox: Don’t select active item if it’s hidden

I hope no one ever actually brings such a silly item into this world,
but this achieves symmetry with the similar checks immediately after.

combobox: Move variables into narrowest scopes

Update Slovak translation

vulkan: Fix copy/paste error

vulkan: Add compiled shaders

I forgot to include them when adding the shaders.

vulkan: Compute right size for outset shadow

We were emitting the box and not the shadow box as the rectangle. So we
didn't draw any shadows. Oops.

css: Remove deprecated number-as-pixels compatibility

Previously, for compatibility with GTK 3.0, we allowed specifying
numbers without units and interpreted them as pixels, even when the CSS
specification didn't.

Remove that now that we can break API.

vulkan: Add support for unblurred box-shadow

rendernode: Add getters for box-shadow nodes

docs: Remove mention of GdkColor

Revert "filechooser portal: Make sure we can save"

This reverts commit 4875c689a0e826604953965ffdb71d8b82cae364.

This was a thinko. Writable is not actually settable from the
application side, but only for the user, from the backend side.

filechooser portal: Make sure we can save

Explicitly request files for saving to be writable, otherwise
we are at the mercy of the portal, which currently gets it wrong.

gdk/wayland: Always get the seat's key modifiers from the GdkKeymap

Elsewhere we already go through the keymap to get modifiers so we
should do the same here. In fact, this was relying on xkb modifier
mask values being bitwise compatible with GdkModifierType which isn't
necessarily true.

https://bugzilla.gnome.org/show_bug.cgi?id=770112

gdk/wayland: Don't add GDK_META_MASK to gdk mod masks if MOD1 is set

Gtk+ treats MOD1 as a synonym for Alt, and does not expect it to be
mapped around, so we should avoid adding GDK_META_MASK if MOD1 is
already included to avoid confusing gtk+ and applications that rely on
that behavior.

https://bugzilla.gnome.org/show_bug.cgi?id=770112

wayland: Handle subsurface as popup parent

When a subsurface is used as a parent of a popup, GDK needs to traverse
up to the transient-for as the next parent, to properly find the parent
used by the popup positioner. This is because the parent of a popup
must always either be an xdg_popup or an xdg_surface, but traversing
the "parent" (in GDK terms) upwards from a subsurface will end up on
the fake root window before we hit the actual parent (in Wayland terms).

https://bugzilla.gnome.org/show_bug.cgi?id=776225

vulkan: Add fallback message for node-as-texture

This is mainly to motivate me to implement it.

vulkan: Generate clip shaders from same source

Instead of having 3 different shaders for the different clipping
versions, just have one shader and use a preprocessor define to use
different clip functions.

That preprocessor define is set in the Makefile.

Also use foo.frag and foo.vert as the file extensions instead of using
foo.frag.glsl and foo.vert.glsl, as that's what glslc suggests as
extension.

vulkan: Add clip.vert.glsl

Implement clipping the same way as in the last commit for the
fragment shaders.

vulkan: Add a clip.frag.glsl include

This include is supposed to handle clipping for the different clipping
methods.

So far, we only use it in the rounded rect cases.

vulkan: Add push constants to fragment shader

That way we don't need to move the clip rounded rect manually through
the vertex shader into the fragment shader but can just look at the push
constants.

Simplifies shaders a lot.

vulkan: Move push constants into their own header

This is the first step towards easing maintenance of the Vulkan shaders
by moving common code into headers.

wayland: avoid an unnecessary g_list_length call

https://bugzilla.gnome.org/show_bug.cgi?id=777363

3.89.3

We no longer have gdktestutils

Update the docs build to reflect that.

Updated Spanish translation

Updated Spanish translation

switch: Fix build

Partial revert of 2b6b5c1c8fecdd66172516812ed3f1825619dafc, which
removed a little too much.

switch: Remove some unused code

gtkentryprivate: Remove unused prototype

scalebutton: Use GtkButon:icon-name in the ui file

spinbutton: Restore gtk3 focus behavior

Focus the spinbutton on button press, never focus the buttons.

inspector: Use GtkButton:icon-name in ui files

entry: Remove unused struct member

gtk4-section: Add GtkInfoBar:revealed setter+getter